Add a 200m/400m lane draw converter - #68
Draft
jchen1 wants to merge 2 commits into
Draft
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Bundle Size
|
Replaces the 200m-only converter with one that covers both sprints in lanes for both sexes. Each event and sex gets its own lane slope, pooled by inverse-variance weighting from a Diamond League estimate (2015-2025) and an NCAA estimate (2016-2026), and the pooled confidence interval drives an uncertainty range under the converted time. The old slug redirects to the new one with its query params intact, and stays out of the utilities index via a `hidden` export that getAllPages() filters on. Unlike its siblings, the page waits for router.isReady before writing the URL so that a shared link's params survive a cold load. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Replaces this page's own router.isReady adoption gate with the hook the other calculators use, so all five restore state from a shared link the same way. Enum-ish params are validated before they reach state: event and sex against their unions, lanes against the selectable lanes. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
jchen1
force-pushed
the
jchen1/lane-converter-v2
branch
from
August 12, 2026 16:59
2d526d2 to
5642ece
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Extends the lane-draw converter from a single hardcoded 200m-men constant to all four sprint cohorts, using new fixed-effects estimates from Diamond League (2015–2025) and NCAA FlashResults (2016–2026) data.
/projects/track/lane-draw/: event (200m/400m) and sex selects; conversion shows an uncertainty range derived from the pooled 95% CI (hidden when current lane == target lane).lib/track/laneDraw.ts: pooled inverse-variance constants per event×sex — 200m men −0.0232, 200m women −0.0125, 400m men −0.0240, 400m women −0.0356 s/lane — with all eight source estimates (elite + collegiate, athlete+race FE) documented alongside./projects/track/200m-lane-draw/becomes a param-preserving client redirect (static export → client-side), hidden from the utilities index and search index via a newhiddenpage export.router.isReadygate) before writing defaults back to the URL, so Share links restore state.Testing
npm run buildpasses (202 pages);tsc --noEmit, eslint, prettier clean.npm start): conversion arithmetic hand-verified for all four cohorts, URL params round-trip, old-slug redirect carries params and converts identically, utilities/search indices list the new page once and the old slug not at all, sibling calculators unaffected.🤖 Generated with Claude Code